Skip to content

Conversation

TheYoctoJester
Copy link
Contributor

The mender-connect.service unit should be installed if the resulting distribution is using systemd, and not only if the mender-systemd flag is set in MENDER_FEATURES.

Otherwise installing mender-connect is broken in cases where no full Mender integration is present.

External Contributor Checklist

🚨 Please review the guidelines for contributing to this repository.

  • Make sure that all commits follow the conventional commit specification for the Mender project.

The majority of our contributions are fixes, which means your commit should have
the form below:

fix: <SHORT DESCRIPTION OF FIX>

<OPTIONAL LONGER DESCRIPTION>

Changelog: <USER-FRIENDLY-CHANGE-DESCRIPTION> or <None>
Ticket: <TICKET NUMBER> or <None>
  • Make sure that all commits are signed with git --signoff. Also note that the signoff author must match the author of the commit.

Description

Please describe your pull request.

Thank you!

@TheYoctoJester TheYoctoJester requested a review from a team as a code owner July 15, 2025 09:05
@mender-test-bot
Copy link

@TheYoctoJester, start a full client pipeline with:

  • mentioning me and start client pipeline

my commands and options

You can prevent me from automatically starting CI pipelines:

  • if your pull request title starts with "[NoCI] ..."

You can trigger a client pipeline on multiple prs with:

  • mentioning me and start client pipeline --pr mender/127 --pr mender-connect/255

You can trigger GitHub->GitLab branch sync with:

  • mentioning me and sync

You can cherry pick to a given branch or branches with:

  • mentioning me and:
 cherry-pick to:
 * 1.0.x
 * 2.0.x

The mender-connect.service unit should be installed
if the resulting distribution is using systemd, and not
only if the `mender-systemd` flag is set in `MENDER_FEATURES`.

Otherwise installing mender-connect is broken in cases
where no full Mender integration is present.

Ticket: None
Changelog: Title

Signed-off-by: Josef Holzmayr <[email protected]>
@TheYoctoJester TheYoctoJester force-pushed the remove_unneccessary_mender_systemd_upstream branch from ada13ec to a9baa2f Compare July 15, 2025 09:08
@mender-test-bot
Copy link

Merging these commits will result in the following changelog entries:

Changelogs

meta-mender (remove_unneccessary_mender_systemd_upstream)

New changes in meta-mender since master:

Bug Fixes
  • mender-connect: fix systemd DISTRO_FEATURES handling

@TheYoctoJester
Copy link
Contributor Author

@mender-test-bot start client pipeline

@mender-test-bot
Copy link

Hello 😺 I created a pipeline for you here: Pipeline-1927558916

Build Configuration Matrix

Key Value
BUILD_BEAGLEBONEBLACK true
BUILD_CLIENT true
BUILD_QEMUX86_64_BIOS_GRUB true
BUILD_QEMUX86_64_BIOS_GRUB_GPT true
BUILD_QEMUX86_64_UEFI_GRUB true
BUILD_VEXPRESS_QEMU true
BUILD_VEXPRESS_QEMU_FLASH true
BUILD_VEXPRESS_QEMU_UBOOT_UEFI_GRUB true
META_MENDER_REV pull/2289/head
RUN_INTEGRATION_TESTS true
TEST_QEMUX86_64_BIOS_GRUB true
TEST_QEMUX86_64_BIOS_GRUB_GPT true
TEST_QEMUX86_64_UEFI_GRUB true
TEST_VEXPRESS_QEMU true
TEST_VEXPRESS_QEMU_FLASH true
TEST_VEXPRESS_QEMU_UBOOT_UEFI_GRUB true

@TheYoctoJester
Copy link
Contributor Author

@mender-test-bot start client pipeline

@mender-test-bot
Copy link

Hello 😺 I created a pipeline for you here: Pipeline-1960767182

Build Configuration Matrix

Key Value
BUILD_BEAGLEBONEBLACK true
BUILD_CLIENT true
BUILD_QEMUX86_64_BIOS_GRUB true
BUILD_QEMUX86_64_BIOS_GRUB_GPT true
BUILD_QEMUX86_64_UEFI_GRUB true
BUILD_VEXPRESS_QEMU true
BUILD_VEXPRESS_QEMU_FLASH true
BUILD_VEXPRESS_QEMU_UBOOT_UEFI_GRUB true
META_MENDER_REV pull/2289/head
RUN_INTEGRATION_TESTS true
TEST_QEMUX86_64_BIOS_GRUB true
TEST_QEMUX86_64_BIOS_GRUB_GPT true
TEST_QEMUX86_64_UEFI_GRUB true
TEST_VEXPRESS_QEMU true
TEST_VEXPRESS_QEMU_FLASH true
TEST_VEXPRESS_QEMU_UBOOT_UEFI_GRUB true

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants